Skip to content
This repository was archived by the owner on Apr 12, 2026. It is now read-only.

Added NotificationV2 and Group Events to Pipeline Tutorial#134

Merged
Foorack merged 2 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:main
Mar 14, 2023
Merged

Added NotificationV2 and Group Events to Pipeline Tutorial#134
Foorack merged 2 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:main

Conversation

@VinyarionHyarmendacil
Copy link
Copy Markdown
Contributor

The following pipeline events are mostly documented, barring the properties specified:

NotificationV2 Events:

  • notification-v2
    • $.content.type: incomplete enum
    • $.content.category: incomplete enum
    • $.content.relatedNotificationsId: not sure what a gpos is
    • $.content.link: unknown scheme values?
    • $.content.responses[].type: incomplete enum
    • $.content.responses[].data: unknown formats depending on type
    • $.content.responses[].icon: incomplete enum
  • notification-v2-update
  • notification-v2-delete

Group Events:

  • group-joined
  • group-left
  • group-member-updated
    • $.content.member.membershipStatus: incomplete enum
    • $.content.member.visibility: incomplete enum
  • group-role-updated

NotificationV2 Events:
  notification-v2
  notification-v2-update
  notification-v2-delete
Group Events:
  group-joined
  group-left
  group-member-updated
  group-role-updated
@ariesclark ariesclark requested review from Foorack and Rexios80 March 11, 2023 03:10
Copy link
Copy Markdown
Member

@Foorack Foorack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you extremely much for this @VinyarionHyarmendacil

LGTM!

@Rexios80
Copy link
Copy Markdown
Contributor

I don't know enough about this to comment. If it looks good to @Foorack it's good for me.

@Foorack Foorack merged commit 6a395a3 into vrchatapi:main Mar 14, 2023
@VinyarionHyarmendacil
Copy link
Copy Markdown
Contributor Author

#129 addressed, forgot to refer earlier

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants